⚠️ No Changeset found

Latest commit: 7b5682c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
thirdweb-www	Ready	Preview	Comment	Nov 19, 2025 7:44pm

4 Skipped Deployments

Project	Deployment	Preview	Comments	Updated (UTC)
docs-v2	Skipped			Nov 19, 2025 7:44pm
nebula	Skipped			Nov 19, 2025 7:44pm
thirdweb_playground	Skipped			Nov 19, 2025 7:44pm
wallet-ui	Skipped			Nov 19, 2025 7:44pm

Walkthrough

When canClaimFaucetQuery indicates paid-plan-required, FaucetButton now renders an upsell link/button to /billing with an inline arrow icon, short-circuiting the previous disabled-button path and removing the prior inline availability message; other disabled cases (throttle/unsupported-chain) are unchanged.

Changes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant FaucetButton
  participant Billing as "/billing"

  User->>FaucetButton: open/interact
  FaucetButton->>FaucetButton: check canClaimFaucetQuery.data
  alt paid-plan-required
    FaucetButton->>User: render upsell link/button (arrow icon)
    User->>Billing: follow link to /billing
  else throttle or unsupported-chain
    FaucetButton->>User: render disabled button with reason
  end

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Files to check closely:
  • apps/.../FaucetButton.tsx — verify import, link URL /billing, accessibility (aria) attributes, visual/icon alignment, and that other disabled branches remain intact.

Pre-merge checks and finishing touches

Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.

Learnt from: saminacodes
Repo: thirdweb-dev/js PR: 7543
File: apps/portal/src/app/pay/page.mdx:4-4
Timestamp: 2025-07-07T21:21:47.488Z
Learning: In the thirdweb-dev/js repository, lucide-react icons must be imported with the "Icon" suffix (e.g., ExternalLinkIcon, RocketIcon) as required by the new linting rule, contrary to the typical lucide-react convention of importing without the suffix.

Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7768
File: apps/playground-web/src/app/navLinks.ts:1-1
Timestamp: 2025-07-31T16:17:42.753Z
Learning: Configuration files that import and reference React components (like icon components from lucide-react) need the "use client" directive, even if they primarily export static data, because the referenced components need to be executed in a client context when used by other client components.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

• [PRO-106] Dashboard: Update Bridge stats #8430
• Link to team billing page in faucet button on free plan #8429 👈 (View in Graphite)
• main

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• merge-queue - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.83%. Comparing base (51f46d3) to head (7b5682c).
⚠️ Report is 2 commits behind head on main.

@@           Coverage Diff           @@
##             main    #8429   +/-   ##
=======================================
  Coverage   54.83%   54.83%           
=======================================
  Files         919      919           
  Lines       60868    60868           
  Branches     4141     4141           
=======================================
  Hits        33375    33375           
  Misses      27391    27391           
  Partials      102      102           

size-limit report 📦

Merge activity

• Nov 18, 4:50 PM UTC: MananTank added this pull request to the Graphite merge queue.
• Nov 18, 4:58 PM UTC: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'Vercel – thirdweb-www').
• Nov 19, 7:33 PM UTC: MananTank added this pull request to the Graphite merge queue.
• Nov 19, 7:45 PM UTC: Merged by the Graphite merge queue.